Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Xcode 15: fixed release build #3034

Merged
merged 1 commit into from
Aug 17, 2023
Merged

Xcode 15: fixed release build #3034

merged 1 commit into from
Aug 17, 2023

Conversation

NachoSoto
Copy link
Contributor

Fixes #3028 (comment)
This was wrong in #3032 because Purchases.storeKit2Setting is only available in DEBUG builds, but we didn't catch it because we have no CI for Xcode 15 yet, which this adds as well.

Fixes #3028 (comment)
This was wrong in #3032 because `Purchases.storeKit2Setting` is only available in `DEBUG` builds, but we didn't catch it because we have no CI for Xcode 15 yet, which this adds as well.
Copy link
Member

@joshdholtz joshdholtz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@NachoSoto NachoSoto merged commit 91890ed into main Aug 17, 2023
@NachoSoto NachoSoto deleted the fix-xcode-15-release-build branch August 17, 2023 03:44
NachoSoto pushed a commit that referenced this pull request Aug 17, 2023
**This is an automatic release.**

### Bugfixes
* `Xcode 15`: fixed release build (#3034) via NachoSoto (@NachoSoto)
NachoSoto added a commit that referenced this pull request Aug 17, 2023
The `build_tv_watch_mac` job already builds on Release mode (`Carthage`'s default).
This would have also helped detect the issue fixed by #3034.

I've also changed the API tester jobs to compile on release, though those aren't ran by CI.
NachoSoto added a commit that referenced this pull request Aug 22, 2023
The `build_tv_watch_mac` job already builds on Release mode
(`Carthage`'s default).
This would have also helped detect the issue fixed by #3034.

I've also changed the API tester jobs to compile on release, though
those aren't ran by CI.
MarkVillacampa pushed a commit that referenced this pull request Sep 6, 2023
Fixes
#3028 (comment)
This was wrong in #3032 because `Purchases.storeKit2Setting` is only
available in `DEBUG` builds, but we didn't catch it because we have no
CI for Xcode 15 yet, which this adds as well.
MarkVillacampa pushed a commit that referenced this pull request Sep 6, 2023
**This is an automatic release.**

### Bugfixes
* `Xcode 15`: fixed release build (#3034) via NachoSoto (@NachoSoto)
MarkVillacampa pushed a commit that referenced this pull request Sep 6, 2023
The `build_tv_watch_mac` job already builds on Release mode
(`Carthage`'s default).
This would have also helped detect the issue fixed by #3034.

I've also changed the API tester jobs to compile on release, though
those aren't ran by CI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants